Public trips - Rename fields in host request by nabramow · Pull Request #7740 · Couchers-org/couchers

nabramow · 2026-01-25T04:40:59Z

Describe briefly what this PR is doing and why.

To prepare for public trips, we will first rename surfer_user_id and host_user_id to recipient and initiator. This is because public trips will allow us going both directions so the surfer initiates normally but host initiates with public trips.

Testing

Explain how you tested this PR and give clear steps so the reviewer can replicate.

Upgrade and downgrade migrations should work
Send a couple host requests and go through reference flow. Should still work exactly as before.

Backend checklist

Added tests for any new code or added a regression test if fixing a bug
Run the backend locally and it works
Added migrations if there are any database changes, rebased onto develop if necessary for linear migration history

For maintainers

Maintainers can push commits to my branch
Maintainers can merge this PR for me

vercel · 2026-01-25T04:41:04Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
couchers	Ready	Preview	Mar 3, 2026 4:52am

couchersbot · 2026-02-22T04:07:47Z

Claude finished @nabramow's task in 2m 3s —— View job

Migration Safety Review Complete

Get PR diff and identify migration files
Read migration file(s) in full
Check current production schema
Review model changes
Evaluate against safety checklist
Post review summary

Result: SAFE - The migration 0140_rename_host_request_user_columns_v2.py performs metadata-only column, index, and constraint renames on host_requests. No data modification, no type changes, no new constraints. Full details in the migration review comment.

couchersbot · 2026-02-22T04:09:55Z

Migration Review

One of our most common issues is migrations that don't work with existing data in production. This is very hard to catch with CI, so we have this bot to check for potential problems.

Overall Risk: SAFE

Files Reviewed

0140_rename_host_request_user_columns_v2.py - Renames 11 columns, 4 indexes, and 2 FK constraints on host_requests table from surfer/host naming to initiator/recipient naming.

Findings

No issues found. All operations are metadata-only in PostgreSQL:

ALTER COLUMN ... RENAME TO renames are instant metadata updates, no table rewrite
ALTER INDEX ... RENAME TO and ALTER TABLE ... RENAME CONSTRAINT are similarly metadata-only
The user_response_rates materialized view references host_requests columns; PostgreSQL automatically updates view definitions when referenced columns are renamed (tracked by attnum), so the view continues to work correctly
The composite index ix_host_requests_status_reminder_counts references columns by attnum and remains functional after renames

The downgrade function properly reverses all renames in the correct reverse order.

Summary

This is a safe, metadata-only migration. All 11 column renames, 4 index renames, and 2 FK constraint renames are correctly mapped and match the model changes in host_requests.py. No data modification, no type changes, no constraint additions — just naming updates.

couchersbot · 2026-02-22T15:55:35Z

Claude finished @nabramow's task in 1m 49s —— View job

Migration Safety Review Complete

Get PR diff and identify migration files
Read migration file(s) in full
Check current production schema
Review model changes
Evaluate against safety checklist
Post review summary

Result: SAFE - The migration 0140_rename_host_request_user_columns_v2.py performs metadata-only column, index, and constraint renames on host_requests. No data modification, no type changes, no new constraints. Full details in the migration review comment.

aapeliv

some comments. let's just change the column names in the host request table in this PR

app/backend/src/couchers/migrations/versions/0140_rename_host_request_user_columns_v2.py

aapeliv · 2026-02-22T15:57:24Z

app/backend/src/tests/test_moderation.py



-def create_test_host_request_with_moderation(surfer_token, host_user_id):
+def create_test_host_request_with_moderation(surfer_token, recipient_user_id):


let's not change this one

app/backend/src/tests/test_references.py

app/proto/requests.proto

… na/backend/naming-host-req

couchersbot · 2026-03-02T05:27:17Z

Claude finished @nabramow's task in 1m 36s —— View job

Migration Safety Review Complete

Get PR diff and identify migration files
Read migration file(s) in full
Check current production schema
Review model changes
Evaluate against safety checklist
Post review summary

Result: SAFE - The migration 0140_rename_host_request_user_columns_v2.py performs metadata-only column, index, and constraint renames on host_requests. No data modification, no type changes, no new constraints. Full details in the migration review comment.

couchersbot · 2026-03-02T05:30:19Z

Claude finished @nabramow's task in 1m 40s —— View job

Migration Safety Review Complete

Get PR diff and identify migration files
Read migration file(s) in full
Check current production schema
Review model changes
Evaluate against safety checklist
Post review summary

Result: SAFE - The migration 0140_rename_host_request_user_columns_v2.py performs metadata-only column, index, and constraint renames on host_requests. No data modification, no type changes, no new constraints. Full details in the migration review comment.

couchersbot · 2026-03-03T04:47:19Z

Claude finished @nabramow's task in 1m 32s —— View job

Migration Safety Review Complete

Get PR diff and identify migration files
Read migration file(s) in full
Check current production schema
Review model changes
Evaluate against safety checklist
Post review summary

Result: SAFE - The migration 0140_rename_host_request_user_columns_v2.py performs metadata-only column, index, and constraint renames on host_requests. No data modification, no type changes, no new constraints. Full details in the migration review comment.

Rename fields

f3b9019

vercel bot deployed to Preview January 25, 2026 04:41 View deployment

Forgot to save these

d11d8c4

vercel bot had a problem deploying to Preview January 25, 2026 13:10 Failure

nabramow mentioned this pull request Feb 16, 2026

Backend: Add public_trips models and migration #7723

Merged

7 tasks

Update frontend keys

ebe3138

nabramow marked this pull request as ready for review February 22, 2026 04:07

nabramow requested a review from aapeliv as a code owner February 22, 2026 04:07

vercel bot deployed to Preview February 22, 2026 04:13 View deployment

nabramow changed the title ~~WIP DRAFT - Rename fields in host request~~ Public trips - Rename fields in host request Feb 22, 2026

Merge branch 'develop' into na/backend/naming-host-req

a067b1f

vercel bot deployed to Preview February 22, 2026 15:58 View deployment

aapeliv requested changes Feb 22, 2026

View reviewed changes

nabramow added 2 commits March 2, 2026 12:26

Undo changes that affect frontend

55aaf03

Merge remote-tracking branch 'origin/na/backend/naming-host-req' into…

079b322

… na/backend/naming-host-req

Merge branch 'develop' into na/backend/naming-host-req

8945e07

vercel bot had a problem deploying to Preview March 2, 2026 05:35 Failure

Merge branch 'develop' into na/backend/naming-host-req

0e1b948

vercel bot deployed to Preview March 3, 2026 04:52 View deployment

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Public trips - Rename fields in host request#7740

Public trips - Rename fields in host request#7740
nabramow wants to merge 8 commits intodevelopfrom
na/backend/naming-host-req

nabramow commented Jan 25, 2026

Uh oh!

vercel bot commented Jan 25, 2026 •

edited

Loading

Uh oh!

couchersbot bot commented Feb 22, 2026 •

edited

Loading

Uh oh!

couchersbot bot commented Feb 22, 2026 •

edited

Loading

Uh oh!

couchersbot bot commented Feb 22, 2026 •

edited

Loading

Uh oh!

aapeliv left a comment

Uh oh!

Uh oh!

aapeliv Feb 22, 2026

Uh oh!

Uh oh!

Uh oh!

couchersbot bot commented Mar 2, 2026 •

edited

Loading

Uh oh!

couchersbot bot commented Mar 2, 2026 •

edited

Loading

Uh oh!

couchersbot bot commented Mar 3, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants



		def create_test_host_request_with_moderation(surfer_token, host_user_id):
		def create_test_host_request_with_moderation(surfer_token, recipient_user_id):

Conversation

nabramow commented Jan 25, 2026

Testing

For maintainers

Uh oh!

vercel bot commented Jan 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

couchersbot bot commented Feb 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Migration Safety Review Complete

Uh oh!

couchersbot bot commented Feb 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Migration Review

Files Reviewed

Findings

Summary

Uh oh!

couchersbot bot commented Feb 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Migration Safety Review Complete

Uh oh!

aapeliv left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

aapeliv Feb 22, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

couchersbot bot commented Mar 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Migration Safety Review Complete

Uh oh!

couchersbot bot commented Mar 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Migration Safety Review Complete

Uh oh!

couchersbot bot commented Mar 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Migration Safety Review Complete

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

vercel bot commented Jan 25, 2026 •

edited

Loading

couchersbot bot commented Feb 22, 2026 •

edited

Loading

couchersbot bot commented Feb 22, 2026 •

edited

Loading

couchersbot bot commented Feb 22, 2026 •

edited

Loading

couchersbot bot commented Mar 2, 2026 •

edited

Loading

couchersbot bot commented Mar 2, 2026 •

edited

Loading

couchersbot bot commented Mar 3, 2026 •

edited

Loading